Onko @ $ array ['mahdollisesti_missing_key'] anti-malli?

@ -Operaattori tukahduttaa virheilmoitukset, ja sen käyttäminen saattaa asettaa koodisi muille virheille ja odottamattomalle käytökselle, jota on vaikea löytää. Siksi se on varmasti antipattern.
Siksi haluaisin erittäin paljon toista bittiä. Se tekee siitä paljon selkeämmän
- että sitä ei välttämättä ole taulukossa, ja
mikä on oletusarvo, jos sitä ei ole
Voit tehdä siitä suppeamman käyttämällä kolminkertaista ehdollista operaattoria?:, Kuten Mark Bakerin vastaus osoittaa. Hieman vähemmän koodia ja enemmän symboleja, mutta merkitys on hyvin tunnistettu.
|
Itse asiassa vaihteluväli on anti-kuvio. Jos käytät vain isset ($ var)? $ Var: NULL -toimintoa tarkoituksenaan estää "virhe", et ole saavuttanut mitään oikean syntaksin käytöstä virheiden estämiseksi. Sillä on sama tulos, mutta se on vähemmän luettavissa.
Ihmiset kiistävät sitä sen vuoksi, että koetaan "puhtaus" ja koska issetin käyttö on mikrooptimointia. @: N välttäminen ja issetin käyttäminen syntaktisena suolan korvauksena on vain rahtikultin ohjelmointi.
|
Tai
$ value = (isset ($ array ['mahdollisesti_missing_key']))? $ array ['mahdollisesti_missing_key']: tyhjä;
|
Varoitusten huomiotta jättäminen on ehdottomasti antipattern; joten kyllä, se on anti-malli (ja voin taata, että jos opit tukahduttamaan varoitukset, yksi heistä palaa takaisin ja puree sinua takaosaan, ellei pahempaa).
Vaikka toinen versio onkin tarkempi, se antaa alustamattomalle muuttujalle tunnetun tilan (tai sitä voidaan käyttää ongelman käsittelemiseen, jos muuttujan on tarkoitus olla täytetty).
|
Kolmas vaihtoehto:
$ value = (isset ($ array ['key'])? $ array ['key']: null);
Tiedän, että tämä ei suoraan vastaa kysymykseen; Olisin kirjoittanut sen kommenttina, paitsi että se todella tarvitsi alustaa.
Ajatuksena on, että jos yrität lyhentää koodiasi käyttämällä yksirivistä if-else-lohkon sijasta, voit silti saada sen ytimekkäänä yksiriviseksi kolmikantaisen operaattorin avulla, mikä antaa sinulle parhaan molempien maailmojen.
|
Toinen koodilohko (tai Mark Bakerin vaihtoehto, joka toimii täsmälleen samalla tavalla) on parempi. En ole täysin varma PHP: stä, mutta monissa muissa ohjelmointikielissä yksinkertaisesti muuttujan ohittaminen heittäisi melkein varmasti virheen. Ainakin toisella lohkolla aloitat muuttujan johonkin arvoon tai muistipaikkaan.
Virheiden estämistä tulisi käyttää yleisemmin, jos oletat, että jokin toiminto heittää odotetun virheen lopputuotteeseen (suurelta osin näin ei kuitenkaan ole).
Onnea!
Dennis M.
|
Vastauksesi
StackExchange.ifUsing ("editor", function () {
StackExchange.using ("externalEditor", function () {
StackExchange.using ("katkelmat", function () {
StackExchange.snippets.init ();
});
});
}, "koodinpätkät");
StackExchange.ready (function () {
var channelOptions = {
tunnisteet: "" .split (""),
id: "1"
};
initTagRenderer ("". split (""), "" .split (""), channelOptions);
StackExchange.using ("externalEditor", function () {
// Sinun on käynnistettävä editori katkelmien jälkeen, jos katkelmat ovat käytössä
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using ("katkelmat", function () {
createEditor ();
});
}
muu {
createEditor ();
}
});
funktio createEditor () {
StackExchange.prepareEditor ({
useStacksEditor: väärä,
heartbeatType: 'vastaus',
autoActivateHeartbeat: epätosi,
convertImagesToLinks: true,
noModals: totta,
showLowRepImageUploadWarning: tosi,
maineToPostImages: 10,
bindNavPrevention: tosi,
postfix: "",
imageUploader: {
brandingHtml: "Palvelun tarjoaa \ u003ca href = \" https: //imgur.com/ \ "\ u003e \ u003csvg class = \" svg-icon \ "width = \" 50 \ "height = \" 18 \ "viewBox = \ "0 0 50 18 \" fill = \ "none \" xmlns = \ "http: //www.w3.org/2000/svg \" \ u003e \ u003cpath d = \ "M46.1709 9.17788C46.1709 8.26454 46.2665 7.94324 47.1084 7.58816C47.4091 7.46349 47.7169 7.36433 48.0099 7.26993C48.9099 6.97997 49.672 6.73443 49.672 5.93063C49.672 5.22043 48.9832 4.61182 48.1414 4.61182C47.4335 4.61182 46.7256 4.932 43.1481 6.59048V11.9512C43.1481 13.2535 43.6264 13.8962 44.6595 13.8962C45.6924 13.8962 46.1709 13.2535 46.1709 11.9512V9.17788Z \ "/ \ u003e \ u003cpath d = \" M32.492 14.44.44 14.44 144 144 144 144 044 144 044 144 044 144 044 144 044 144 044 144 044 144 044 144 044 144 0 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 144 141 141 144 14 14 14 14 14 14 14 14 14 14 41.5985 12.6954 41.5985 10.1419V6.59049C41.5985 5.28821 41.1394 4.66232 40.1061 4.66232C39.0732 4.66232 38.5948 5.28821 38.5948 6.59049V9.60062C38.5948 10.8521 38.2696 11.5455 37.0451 11.5455C35 521 35.4954 9.60062V6.59049C35.4954 5.28821 35.0173 4.66232 34.0034 4.66232C32.9703 4.66232 32.492 5.28821 32.492 6.59049V10.1419Z \ "/ \ u003e \ u003cpath fill-rule = \" evenodd \ "clip-rule = = \ "M25.6622 17.6335C27.8049 17.6335 29.3739 16.9402 30.2537 15.6379C30.8468 14.7755 30.9615 13.5579 30.9615 11.9512V6.59049C30.9615 5.28821 30.4833 4.66231 29.4502 4.66231C28.995 4 439 4 264 4 464 4 950 4 265 4 432 4 4 492 4 4 492 4 4 492 435 4 435 4 432 435 4 432 435 4 432 435 4 435 4 432 4 462 435 4 439 4 269 4 269 4 269 263 4 269 263 4 269 265 4 262 4 9 5 9 .1369 4,56087 21,01346,57349 21,0134 9.27932C21.0134 11,9852 23,003 13,913 25,3754 13.913C26.5612 13,913 27,4607 13,4902 28,1109 12.6616C28.1109 12,7229 28,1161 12,7799 28,121 12.8346C28.1256 12,8854 28,1301 12,9342 28,1301 12.983C28.1301 14,4373 27,2502 15,2321 25,777 15.2321C24.8349 15,2321 24,1352 14,9821 23,5661 14.7787C23.176 14,6393 22,8472 14,5218 22,5437 14.5218C21.7977 14,5218 21,2429 15,0123 21,2429 15.6887C21.2429 16,7375 22,9072 17,6335 25,6622 17.6335ZM24.1317 9.27932C24.1317 7,94324 24,9928 7,09766 26,1024 7.09766C27.2119 7,09766 28,0918 7,94324 28,0918 9.27932C28.0918 10,6321 27,2311 11.5116 26.1024 11.5116C24.9737 11.5116 24.1317 10.6491 24.1317 9.27932Z \ "/ \ u003e \ u003cpath d = \" M16.8045 11.9512C16.8045 13.2535 17.2637 13.8962 18.2965 13.8962C19.3298 13.8591980 13.85919 1913 4.62866 16.4027 4.62866C15.1594 4.62866 14.279 4.98375 13.3609 5.88013C12.653 5.05154 11.6581 4.62866 10.3573 4.62866C9.34336 4.62866 8.57809 4.89931 7.9466 5.5079C7.58314 4. 9328 7.10506 4.66232 6.51203 4.66232C5.47873 4.66232 5.00066 5.28821 5.00066 6.59049V11.9512C5.00066 13.2535 5.47873 13.8962 6.51203 13.8962C7.54479 13.8962 8.0232 13.2535 8.0232 11.95124119119119119119119 .9512C10.893 13,2535 11,3711 13,8962 12,4044 13.8962C13.4375 13,8962 13,9157 13,2535 13,9157 11.9512V8.90741C13.9157 7,58817 14,3365 6,91179 15,4269 6.91179C16.4027 6,91179 16,8045 7,58817 16,8045 8.94108V11.9512Z \ "/ \ u003e \ u003cpath d = \" M3 0,31675 6.59049C3.31675 5,28821 2,83866 4,66232 1,82471 4.66232C0.791758 4,66232 0,313354 5,28821 0,313354 6.59049V11.9512C0.313354 13,2535 0,791758 13,8962 1,82471 13.8962C2.85798 13,8962 3,31675 13,2535 3,31675 11.9512V6.59049Z \"/ \ u003e \ u003cpath d = \ "M1.87209 0.400291C0.843612 0.400291 0 1.1159 0 1.98861C0 2.87869 0.822846 3.57676 1.87209 3.57676C2.90056 3.57676 3.7234 2.87869 3.7234 1.98861C3.7234 1.1159 2.90056 0.400291 \ ZB 1B = 1" \ u003c / svg \ u003e \ u003c / a \ u003e ",
contentPolicyHtml: "Käyttäjän käyttöoikeudet lisensoitu kohtaan \ u003ca href = \" https: //stackoverflow.com/help/licensing \ "\ u003ecc by-sa \ u003c / a \ u003e \ u003ca href = \" https://stackoverflow.com / legal / content-policy \ "\ u003e (sisältökäytäntö) \ u003c / a \ u003e",
allowUrls: totta
},
onDemand: totta,
discardSelector: ".discard-answer"
, välittömästiShowMarkdownHelp: true, enableTables: true, enableSnippets: true
});
}
});
Kiitos vastauksestasi pinon ylivuotoon!
Muista vastata kysymykseen. Anna yksityiskohdat ja jaa tutkimuksesi!
Mutta vältä…
Pyydä apua, selvennystä tai vastaamista muihin vastauksiin.
Lausuntojen tekeminen mielipiteen perusteella; varmuuskopioi ne viitteillä tai henkilökohtaisella kokemuksella.
Jos haluat lisätietoja, katso vinkit hyvien vastausten kirjoittamiseen.
Luonnos tallennettu
Luonnos hylätty
Rekisteröidy tai kirjaudu sisään
StackExchange.ready (function () {
StackExchange.helpers.onClickDraftSave ('# login-link');
});
Rekisteröidy Googleen
Rekisteröidy Facebookilla
Rekisteröidy sähköpostilla ja salasanalla
Lähetä
Lähetä vieras
Nimi
Sähköposti
Pakollinen, mutta ei koskaan näytetty
StackExchange.ready (
funktio () {
StackExchange.openid.initPostLogin ('. New-post-login', 'https% 3a% 2f% 2fstackoverflow.com% 2fquestions% 2f4441528% 2fis-arraypossible-missing-key-an-anti-pattern% 23new-answer', ' kysymyssivu ');
}
);
Lähetä vieras
Nimi
Sähköposti
Pakollinen, mutta ei koskaan näytetty
Lähetä vastauksesi
Hylkää
Klikkaamalla "Lähetä vastauksesi" hyväksyt käyttöehdot, tietosuojakäytännön ja evästekäytännön
Eikö vastausta etsit? Selaa muita kysymyksiä, jotka on merkitty php-matriisien operaattoreiden virheiden torjunnassa tai kysy omia kysymyksiä.